Skip to content

perf: add ncbi api key option in .env files#38

Merged
TomHarrop merged 6 commits into
mainfrom
speed-up-ncbi-lookup
Jul 22, 2026
Merged

perf: add ncbi api key option in .env files#38
TomHarrop merged 6 commits into
mainfrom
speed-up-ncbi-lookup

Conversation

@keeva-c

@keeva-c keeva-c commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

📌 Summary

Adding an option to include an NCBI API key in the .env file to improve rate limiting.

🔄 Type of Change

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • ✨ New feature (non-breaking change that adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to change)
  • 🧹 Refactor (code change that neither fixes a bug nor adds a feature)
  • 📚 Documentation update
  • ⚙️ CI / build / tooling change

🧩 Key Changes

  • adding an option for using an NCBI API key for taxonomic lookups
  • updating key name to ncbi_api_key
  • adding a guard to only apply the key in the API request parameters if a key has been provided

✅ Checklist

  • Tests are passing locally / in CI
  • New tests have been added for new or changed functionality
  • Self-review completed
  • Documentation updated (if needed)
  • No sensitive data or secrets included

🔍 Review Notes


📎 Related Issues / Tickets

  • Closes #
  • Related to #

@TomHarrop
TomHarrop merged commit 0d271aa into main Jul 22, 2026
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants